The current flickr field requires three user entries: type (photo or photoset), flickr user id and photo/photoset id.
I wonder why the flickr user id (nsid) is needed, it is not used in the source code (it is only written, not read). Moreover, the nsid can be inferred from the photo/photo set id (e.g. with flickr_photo_get_info()), so the nsid is redundant information and it hurts the usability of the field.

In addition, a client of me wanted a field for Flickr photo sets and didn't want to be bothered with option between photo/photoset.

In attachment I have a patch that:

  • Adds a new CCK field "flickrfield_photoset" to store references to Flickr photo sets to flickrfield.module.
  • field only stores the photo set id (no nsid required).
  • widget: only asks the photo set id (see attached screenshot)
  • formatter options: primary photo from set in different sizes and an embedded Flickr.com flash slideshow (see attached screenshot)
CommentFileSizeAuthor
#7 409694_flickrfieldphotoset.patch10.36 KBsoxofaan
#7 flickrfielformatters.png73.97 KBsoxofaan
#7 simplelinkexample.png51.87 KBsoxofaan
flickrfield_view.png370.29 KBsoxofaan
flickrfield_edit.png44.1 KBsoxofaan
flickrfield_admin.png81.57 KBsoxofaan
flickrfield_photosetfield.patch7.33 KBsoxofaan

Comments

Anonymous’s picture

Anonymous (not verified) commented

This looks awesome. I'll take a look at this later today.

Best, Paul

Anonymous’s picture

Anonymous (not verified) commented
Title: Flickr field for photo set with Flickr.com flash slideshow formatter » [NEEDS REVIEW] Flickr field for photo set with Flickr.com flash slideshow formatter
Assigned: Unassigned »
soxofaan’s picture

soxofaan commented

Hi Paul,

can you give some timeframe about when this patch could be committed?
I want to use it on a production site, but I'm holding it off as long as the patch is in review mode.

Or are you okay with the concept of the patch, but haven't found time to review it thoroughly enough?
If it's only about some details, I could start using it and upgrade later to the final version.

thanks for your time and work anyway.

bennebiest’s picture

bennebiest commented

Hi,

I patched the flickr module using cygwin on a clean Drupal 6 install. (only CCK, Flickrfield and Flickr module installed)
After patching I got this error:

   * warning: include_once(./modules/flickr/field/flickrfield.module) [function.include-once]: failed to open stream: Permission denied in C:\wamp\www\freshone\includes\bootstrap.inc on line 611.
   * warning: include_once() [function.include]: Failed opening './modules/flickr/field/flickrfield.module' for inclusion (include_path='.;C:\php5\pear') in C:\wamp\www\freshone\includes\bootstrap.inc on line 611.
   * warning: Invalid argument supplied for foreach() in C:\wamp\www\freshone\modules\cck\includes\content.admin.inc on line 604.

I'm on WinXP SP3 with WAMP, using Drupal 6.10 with the latest Flickr module.
I Checked ntfs permissions on the Drupal directory, SYSTEM has full control.

Is this causing the patch or could it be my mistake?

thx,
Ben

bcobin’s picture

bcobin commented

Absolutely magnificent! Great work - I'll come back here and post a link when it's deployed on the production site (probably by tomorrow.)

Great documentation, too. A+ all around!

One thing of note: In Firefox, photos marked "private" on Flickr don't load - which is as it should be. On Safari, all photos load, regardless of whether they're private or not. It's probably a browser issue (haven't even tried it in IE yet), but I thought I'd mention it here.

Really great work - thanks much!

Anonymous’s picture

Anonymous (not verified) commented

I couldn't get the slide show to appear.

It maybe that something had changed on the development branch since this patch was created . I'll look try again when i have some more time.

If someone could confirm this patch works against the current development branch i will get back to this as soon as possible

Best, Paul

soxofaan’s picture

soxofaan commented
StatusFileSize
new simplelinkexample.png51.87 KB
new flickrfielformatters.png73.97 KB
new 409694_flickrfieldphotoset.patch10.36 KB

I rerolled the patch against DRUPAL-6--1

and I added in the meantime some more formatters: apart from the flickr sizes, there is also the option to link to the node, link to the Flickr.com photo set page, or no link.
I also added a simple formatter that just shows the url to the Flickr.com photo set page

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Reviewed & tested by the community

Patch still applies, with a little offset. Works very nicely.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Would love to see this for photo Groups as well...

ksenzee’s picture

ksenzee
she/her
commented
Title: [NEEDS REVIEW] Flickr field for photo set with Flickr.com flash slideshow formatter » Flickr field for photo set with Flickr.com flash slideshow formatter

This is a great addition. Committed to DRUPAL-6--1. Thanks very much for the patch and the reviews.

@bcobin, I'm assuming the browser bug you mention is in the Flickr slideshow? If so, that's a problem on Flickr's end. If not, please file a new issue and let us know how to reproduce it.

ksenzee’s picture

ksenzee
she/her
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

ginsonic’s picture

ginsonic commented
Issue summary: View changes

Hi to all!

I am a very very new Drupal user. I want to add to my site articles with gallery on bottom of text alike https://drupal.org/files/issues/flickrfield_view.png sample.

How can I do this?

Any help will be apreciated. Thank you in advance!

lolandese’s picture

lolandese commented

Hi ginsonic. Welcome to Drupal.

Taken from the module documentation at https://drupal.org/node/2171299:

Field configuration

D7

  • Go to /admin/structure/types
  • Click manage fields
  • Add new field > Flickr Photo or Flickr Photo Set (sub-module Flickr Sets should be enabled to use sets in this field)
  • Save field settings
  • Configure as desired
  • Save settings

Then:

  • Go to /admin/structure/types
  • Click manage display
  • Select the formatter for a Flickr field (a size)
  • Save

Thanks.